x11: Fetch vendor/product identifiers for input devices

These are retrieved from XInput device properties.

https://bugzilla.gnome.org/show_bug.cgi?id=740758

device: Add property/getters for vendor/product identifiers

These are a construct only properties, expected to be filled in from
platform specific code.

https://bugzilla.gnome.org/show_bug.cgi?id=740758

gtk-demo: Make editable cells demo more robust

When removing all rows, trying to add rows would not work
and throw criticals. This is fallout from a recent change
to insert rows at the right position. Fix this by handling
the 'empty model' case separately.

https://bugzilla.gnome.org/show_bug.cgi?id=743157

scrolledwindow: Don't displace invalidation by the widget allocation

This was left unmodified when GtkScrolledWindow was made a windowed widget.
Displacing here by the widget allocation is not necessary anymore, since
we are invalidating the window set at these coordinates.

This patch is a simplification of a previous one by Timm Bäder.

https://bugzilla.gnome.org/show_bug.cgi?id=742243

stylecontext: Fix FALSE/TRUE mixup

Introduced in e649c8773f0357df19f4e28aabf40a203a39c409. It's obviously
the root node right there.

scrolled window: Reduce the hover proximity distance further

file chooser: Fix up some focus issues

Better to switch the visible stack child early, since GtkStack
has its own focus handling now.

gtkimagemenuitem: create widget for action icons lazily

GtkImageMenuItem always packs a GtkImage when it is synced with a
related action. This is unnecessary when the action doesn't have an icon
set and results in excess padding when a GtkImageMenuItem is put in a
menu bar (GtkUIManager does this).

Fix this by not creating the GtkImage for icon-less actions.

https://bugzilla.gnome.org/show_bug.cgi?id=742181

render: Replace gtk_style_context_get_border/padding()

Use _gtk_style_context_peek_property() instead.

render: Get rid of gtk_style_context_get_color()

Use _gtk_style_context_peek_property() instead.

Adwaita: Make :active button gradient have 3 stops

Just add a 3rd stop in the place where the gradients for the other
states have a 3rd stop. When running transitions between states, this
avoids cross-fades and instead uses the much faster gradient
transitioning code.

stylecontext: Be more graceful when save() isn't matched with restore()

Instead of asserting, just print a g_warning() and try to work around
the problem.
I hope that g_warning() isn't too spammy for people that are hit with
it.

Also clarify the docs that not restore()ing after a save() is a bad
idea.

https://bugzilla.gnome.org/show_bug.cgi?id=743101

stylecontext: Explicitly specify root node

When creating the query path, explicitly specify if it's for a root node
or for a child. Relying on gtk_style_context_is_saved() is unreliable
(for example when updating the cache).

file chooser: Make search work again

The last round of filechooser redesign work had left
search somewhat dysfunctional. This commit fixes things
up enough that search is once again a working feature
of the file chooser.

file chooser: Drop the location label

Bringing us close to the new design.

file chooser: Don't show 'Enter Location' in save modes

In these modes, the entry is permanently visible anyway.

file chooser: Make Ctrl-L work in recent

Just switch to browse mode when we see Ctrl-L in recent mode.
https://bugzilla.gnome.org/show_bug.cgi?id=660450

file chooser: Ensure the path bar is initialized

There are some cases where we can end up showing the path
bar without ever setting a file on it. One example is:
1) Open file chooser
2) Hit ~
3) Hit Ctrl-L
While this is a bit of a workaround, at least it makes sure
that we never end up with an empty space where the path bar
is supposed to be.

trivial refactoring

file chooser: Make ~ and / work in recent

Complained about in
https://bugzilla.gnome.org/show_bug.cgi?id=743117

Fix a typo in the docs

https://bugzilla.gnome.org/show_bug.cgi?id=743037

broadway: Only use abstract sockets when they are supported

GIO has a function to check whether abstract sockets are
supported, so use it. This should make broadway work on
Freebsd again.

https://bugzilla.gnome.org/show_bug.cgi?id=742980

Make Broadway work on freebsd

It seems that posix_fallocate gives an ENODEV error when
called on an fd opened with shm_open on freebsd. Fix up
the error check to only trigger if we get ENOSPC.

https://bugzilla.gnome.org/show_bug.cgi?id=742980

Fine-tune the proximity hovering for scrollbars

Reduce the distance at which we expand the scrollbar,
and add some hysteresis.

Updated Greek translation

Remove some outdated references

We shouldn't recommend gtk_widget_modify_bg() or
gtk_style_set_background() anymore.

Adjust gtk_status_icon_position_menu() annotation to match GtkMenuPositionFunc again

https://bugzilla.gnome.org/show_bug.cgi?id=674537

https://bugzilla.gnome.org/show_bug.cgi?id=742940

Fix event->button.y_root when using mouse buttons and dnd canceled

Signed-off-by: Maks Naumov <maksqwe1@ukr.net>
https://bugzilla.gnome.org/show_bug.cgi?id=742785

Fix save_hot_y init in gtk_drag_get_icon()

Signed-off-by: Maks Naumov <maksqwe1@ukr.net>
https://bugzilla.gnome.org/show_bug.cgi?id=742786

Updated Basque language

Prefer the Wayland backend

This does not affect X11 sessions, but has the advantage that
we get much more testing of the Wayland backend in Wayland
sessions.

inspector: Remove a bunch of unused variables

stylecontext: Remove a bunch of return_if_fail()s

They're gonna be in the way when refactoring and we can live without
them.

stylecontext: Do not transition from the default style

This gets rid of a bunch of awkward transitions. It's not a perfect
solution to the problem of "should we transition from this state" but it
gets rid of the ugliest offenders.

Adwaita: scrollbar style refactoring...

...making overlay indicator slider visible on any backgroung
in the process.

Updated Czech translation

widget: Free the controller list on finalize()

If this is done on dispose(), the widget may be destroyed (and its
controllers list NULLified) within _gtk_widget_run_controllers(),
causing warnings/crashes when it just tried to hop on the next
controllers.

Freeing the controllers here should be a safety net for implementations,
so it also makes sense to do this late. The widgets that choose to
free their controllers on dispose can still do so, and get
_gtk_widget_remove_controller() called for these as an indirect result.

listbox: Free the multipress gesture on finalize

Updated Icelandic translation

event-controller: Chain up on dispose

Updated Russian translation

gtkplacessidebar: insert bookmark by dnd in correct position

Add needed COLUMN INDEX to "new bookmark" row based on previous
bookmark index plus one.

Fixes bug 742727

Fix accessible_class->widget_set initialization

Signed-off-by: Maks Naumov <maksqwe1@ukr.net>
https://bugzilla.gnome.org/show_bug.cgi?id=742778

Updated Galician translations

Updated Spanish translation

Properly check result of g_utf8_get_char_validated()

g_utf8_get_char_validated() may return -1 or -2
return type is gunichar(guint32)

Therefore such checks like 'gunichar < 0' or 'gunichar > 0'
are always 'false' or 'true'(except when gunichar == 0).

Signed-off-by: Maks Naumov <maksqwe1@ukr.net>
https://bugzilla.gnome.org/show_bug.cgi?id=742774

Fix check for 'y' padding in gtk_cell_renderer_set_padding()

Signed-off-by: Maks Naumov <maksqwe1@ukr.net>
https://bugzilla.gnome.org/show_bug.cgi?id=742764

Remove an unused define

Properly ignore BadWindow and BadDrawable in get_child_info_handler()

Signed-off-by: Maks Naumov <maksqwe1@ukr.net>
https://bugzilla.gnome.org/show_bug.cgi?id=742771

Forgotten file

HighContrast: Don't set nonexisting style properties

These don't exist, so setting them can't do any good.

Adwaita: Don't set nonexisting style properties

These don't exist, so setting them can't do any good.

HighContrast: Also fix separators in file chooser buttons

Same fix as for combo box separators.

Adwaita: Also fix separators in file chooser buttons

Same fix as for combo box separators.

gtkfilechooserwidget: When using an extra widget, show or hide the filter combo as appropriate

https://bugzilla.gnome.org/show_bug.cgi?id=742653

HighContrast: draw combo box separators

Same fix as the previous commit.

Adwaita: draw combo box separators

Killing the separator in the button had the unintended side effect
of also killing separators in the popup.

https://bugzilla.gnome.org/show_bug.cgi?id=742704

widget-factory: Add a combo box separator example

This shows that combo box separators are currently broken
in Adwaita.

GtkTextView: allow to extend text selection with mouse scrollwheel

Fixes bug 710612

file chooser: Add some padidng

https://bugzilla.gnome.org/show_bug.cgi?id=742631

Forgotten files

Adwaita: Fix treeview separators in backdrop

https://bugzilla.gnome.org/show_bug.cgi?id=742702

stylecontext: Stop animations on manual invalidation

If a gtk_style_context_invalidate() is called on a widget's style
context (which nobody should ever do, sheesh!) and we're animating, stop
the animations.

Fixes crashers in Nautilus.

GtkScrolledWindow: Don't start fading right away

When we get a leave event, we need to remove the over bit,
since we may not get motion events that tell us when we lose
proximity, but we don't have to fade out right away - that
will happen on its on due to inactivity a bit later. Having
the indicator stay visible for a bit feels more relaxed.

GtkScrolledWindow: Factor out a constant

Define INDICATOR_CLOSE_DISTANCE instead of hardcoding the same
number of pixels in 4 different places.

GtkScrolledWindow: Don't require proximity for track points

These are hard to navigate precisely, so show wide scrollbars
right away.

GtkScrolledWindow: Drop device manager complications

We no longer use the presence or absence of particular device
types to determine which scrolling to use.

GtkScrolledWindow: Drop a redundant field

The Indicator.enabled field was not carrying any extra information,
so just use priv->use_indicators instead.

GtkScrolledWindow: Rename a private field

touch_mode -> use_indicators, more to the point.

x11/gl: Do not specify the GLX context version

If we use GDK_GL_PROFILE_3_2_CORE we are asking for a core profile
according to the GLX_ARB_create_context_profile extension. For that,
we pass the GLX_CONTEXT_CORE_PROFILE_BIT_ARB value for the
GLX_CONTEXT_PROFILE_MASK_ARB attribute.

The specification for the extension says that:

  If the requested OpenGL version is less than 3.2,
  GLX_CONTEXT_PROFILE_MASK_ARB is ignored and the functionality
  of the context is determined solely by the requested version.

Since we're asking for a core profile, we assume a GL version greater
than or equal to 3.2; thus, we don't need to specify the
GLX_CONTEXT_MAJOR_VERSION_ARB or the GLX_CONTEXT_MINOR_VERSION_ARB
attributes, and instead just rely on whatever version GLX gives us.

This seems to work around a strange issue in Mesa; if we ask for a core
profile and any version > 3.0, we get broken rendering on any shared
context we create.

inspector: Don't use GSettings directly

It is not necessary here, and using GtkSettings gives us
a greater chance to not fail e.g. on Windows.

https://bugzilla.gnome.org/show_bug.cgi?id=742664

inspector: be careful about gsize vs guint64

Since gsize doesn't work as a typename in a ui file,
we have to be careful not to pass pointers to wrongly
sized variables when getting the guint64 values out
of the model.

https://bugzilla.gnome.org/show_bug.cgi?id=742664

docs: Add missing quote

The missing " breaks the syntax highlighting of the GtkImageMenuItem
example.

docs: Add Since for GtkCellRendererPixbuf:surface

docs: Fix some old live.gnome.org URLs

Updated POTFILES.skip

Fix GTK_OVERLAY_SCROLLING=0

This was broken by the recent more dynamic approach.

Fix comment typo

inspector: Show object address

This is useful information when running the application
in a debugger.
https://bugzilla.gnome.org/show_bug.cgi?id=742518

Point to copy/cut/delete named icons in the stock item deprecations

We still use symbolic icons for Copy, Cut, and Delete operations inside
toolbars and some menus, so it's worth pointing to the symbolic icon
names in the deprecation notices.

https://bugzilla.gnome.org/show_bug.cgi?id=741969

file chooser: Don't force a scrollbar

No point in showing the scrollbar when it is not needed.
https://bugzilla.gnome.org/show_bug.cgi?id=742343

gtklabel: Don't accept unhandled mouse buttons

If the pressed mouse button neither triggers a context menu nor
activates a link, set the sequence's state to DENIED.

https://bugzilla.gnome.org/show_bug.cgi?id=742010

cssimage: Fix angle computation for gradients to corners

Updated Basque language

Updated Basque language

stylecontext: Handle theme changes for cached styles

We need to clear the cache manually on full revalidates because
_gtk_css_change_for_child() will clear the full revalidation flags.
And then gtk_style_context_update_cache() will not do the right thing
(which is to clear itself).

stylecontext: Rename GtkStyleInfo to GtkCssNode

stylecontext: update_animating() properly

Because we can switch from animating to non-animating pretty much
anywhere, do the check for animations unconditionally instead of trying
to cram it into the correct if path (and failing).

stack: Remove commented out line

It always shows up in my greps and confuses me.

stylecontext: Add a parent style cache

We now cache the results of lookups on the parent GtkCssStyle. This
allows sharing styles between widgets (recursively). However, this
only works if the styles can't potentially depend on siblings -
neither directly via sibling selectors or via position pseudo-classes
like :first-child.
Unfortunately, Adwaita currently uses first-child a lot, and in
particular for labels, which are the most common widgets.

The big benefits of this change are both less CPU - due to not needing
to compute styles again - and less memory usage - due to sharing of
the styles between widgets. Here's some nonscientific numbers I
collected while pondering the usefulness of this patch:

                    glade   glade   widget
                    demo    demo    factory
                    runtime styles  styles
Adwaita before      19.1s   5800    1150
Adwaita now         18.9s   3800     970
Adwaita hacked now  14.5s   3100     910
simple before       11.3s   5800    1150
simple now          10.8s   1300     590

Adwaita: Adwaita as provided by GTK
Adwaita hacked: Adwaita with the first/last-child for GtkLabel removed
simple: A 250 lines simple GTK theme I use for testing
before: This patch not applied
now: this patch applied

glade demo runtime: Starting glade opening a large file and closing it
glade demo styles: GtkCssStaticStyle objects after opening glade with
                   the large file as per inspector
widget factory styles: GtkCssStaticStyle objects after startup as per
                       inspector

cssstyle: Have a default style

This way we can initialize the default style info with a style and
ddon't need to special case NULL everywhere.

stylecontext: Use correct parent on updates

Previously we looked at the save/restore state when determining the
parent. This is wrong in the case where we're updating the cache though.

So we now save the parent in the style info.

stylecontext: Fix cache updating

After the parent changes in commit
3a337156d11a86c7a88f1f30a09276fdf6f63008 we need to refresh the cached
styles after the current style. After all, they now depend on the base
style.

cssstaticstyle: Handle relevant change

Relevant change is returned from style lookups, so keep is with the
object that holds those values.

stylecontext: Use correct parent style for lookups

After 3a337156d11a86c7a88f1f30a09276fdf6f63008 style lookups still used
the parent context's style as the parent style, even though after a
gtk_style_context_save() the root style of the style context is the
proper parent.

Testcase attached.

cssnodedeclaration: Store the GType of the node

cssstyle: Move function from vfunc to static func

compute_dependencies() is only used internally by GtkCssStaticStyle,
so there's no need to have it elsewhere.

cssstaticstyle: Move function

And with that move, GtkCssStaticStyle is immutable.

cssstaticstyle: Move function

We want the new() return values to be immutable, so we have to move the
code that modifes them.

stylecontext: Refactor update_properties()

It now always returns a new instance.